Skip to content

Conversation

@datalogics-josepha
Copy link
Contributor

@datalogics-josepha datalogics-josepha commented Oct 23, 2025

We have frequent intermittent failures where the Nuget packages are corrupted in some fashion. What we currently do is for each sample build and point our source of Nuget packages at the location on the raid.

I think this is part of the problem, when there's an intermittent hiccup or conflict with what's on the raid.

Instead, let's first copy the packages of interest locally. Then we'll use that as our source to point at.

Also, refactor to remove some redundancy.

This should lead to some performance improvement because now we no longer are dependent on constantly asking dotnet to look at the raid for each sample's build, we just look at the raid once to get what we need and then are local again.

We have frequent intermittent failures where the Nuget packages are corrupted in some fashion.  What we currently do is for each sample build and point our source
of Nuget packages at the location on the raid.

I think this is part of the problem, when there's an intermittent hiccup or conflict with what's on the raid.

Instead, let's first copy the packages of interest locally.  Then we'll use that as our source to point at.

Also, refactor to remove some redundancy.

This should lead to some performance improvement because now we no longer are dependent on constantly asking dotnet to look at the raid for each sample's build,
we just look at the raid once to get what we need and then are local again.
@datalogics-aarroyo
Copy link
Contributor

Looking at the changes. I'm making the assumption that packages to that local directory would get overriden in re-runs. What about when we move to new versions of packages, would that local directory keep all of the previous versions of the packages?

@datalogics-josepha
Copy link
Contributor Author

I think we should be ok in that case, because the Clean Samples step will run in the nightly which will clear out any locally added nupkgs.

@datalogics-aarroyo datalogics-aarroyo merged commit f64f207 into datalogics:main Nov 3, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants